Skip to content

automode to choose from a dropdown list#3857

Open
mattulbrich wants to merge 6 commits into
mainfrom
automodeDropdown
Open

automode to choose from a dropdown list#3857
mattulbrich wants to merge 6 commits into
mainfrom
automodeDropdown

Conversation

@mattulbrich

Copy link
Copy Markdown
Member

Related Issue

n/a

Intended Change

Currently, only the full automation is on a button in the toolbar. But the autopilots have reached a similar degree of relevance and will even more with the JML proof scripts.

Plan

  • Implement a dropdown and adapt calling automation
  • Documented

Type of pull request

New feature (non-breaking change which adds functionality)

A small UI feature that we had discussed already a few times.

Ensuring quality

  • I added new test case(s) for new functionality.

Manually

Additional information and contact(s)

Co-Authored by: kit.qwen3.5-397b-A17b @ KIT, Local AI

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

@mattulbrich mattulbrich added this to the v3.0.0 milestone Jun 22, 2026
@mattulbrich mattulbrich added GUI Feature New feature or request P:LOW labels Jun 22, 2026
@mattulbrich mattulbrich marked this pull request as draft June 22, 2026 16:22
@mattulbrich

Copy link
Copy Markdown
Member Author

Drafting for now, since it was unstable so far and can have one more round of UI testing.

@mattulbrich mattulbrich marked this pull request as ready for review June 22, 2026 20:11
@mattulbrich mattulbrich added the Review Request Waiting for review label Jun 22, 2026
@mattulbrich

Copy link
Copy Markdown
Member Author

As soon as the proof scripts are on the main branch, another button for the script-aware autopilot should be added.

@mattulbrich mattulbrich self-assigned this Jun 22, 2026
@Drodt

Drodt commented Jun 23, 2026

Copy link
Copy Markdown
Member

Can we maybe workshop a better name for the regular automation? "Full Automation" vs. "Full Auto Pilot" is not exactly user friendly.

Perhaps "Apply Strategy"?

Comment thread key.ui/src/main/java/de/uka/ilkd/key/gui/fonticons/IconFactory.java Outdated
@unp1

unp1 commented Jun 23, 2026

Copy link
Copy Markdown
Member

Great idea!

Concerning:

Can we maybe workshop a better name for the regular automation? "Full Automation" vs. "Full Auto Pilot" is not exactly user friendly.

Perhaps "Apply Strategy"?
Just a suggestion:

Maybe sth along the lines

Full Automation (Efficient) <-- former Apply Strategy
Full Automation (Structured) <-- former Full Auto Pilot

One could then have small tooltip explaining

  • Narrow branching, less memory usage, difficult too understand
  • Produces a human-centric proof structure, open goals can be understood easily

@mattulbrich

Copy link
Copy Markdown
Member Author

I like your UI-concerns. Thanks for rasing them.

To unify the concepts: "Apply strategy" should be part of the macro context menu.
The user should not be bothered with the details.

How about the following?

  • Full Automation
  • Structured Automation
  • Structured Automation (Prep.)
  • Script-aware Automation

@mattulbrich

Copy link
Copy Markdown
Member Author

I have updated the UX according to our plan.

* @see de.uka.ilkd.key.macros.FullAutoPilotProofMacro
*/
public static Icon automationFullPilotLogo(int size) {
return automationWithOverlay(size, "F");

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hate to be nit-picky but "S" makes more sense for the structural auto pilot, imo

Suggested change
return automationWithOverlay(size, "F");
return automationWithOverlay(size, "S");

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature or request GUI P:LOW Review Request Waiting for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants